Skip to content

Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto#14182

Merged
acoates-ms merged 5 commits into
microsoft:mainfrom
acoates-ms:visualtoaddto
Dec 9, 2024
Merged

Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto#14182
acoates-ms merged 5 commits into
microsoft:mainfrom
acoates-ms:visualtoaddto

Conversation

@acoates-ms
Copy link
Copy Markdown
Contributor

@acoates-ms acoates-ms commented Dec 9, 2024

Description

Adding ability for custom native components to provide a custom UpdateLayoutMetrics method, and to override VisualToMountChildrenInto.

I also did some cleanup of how override methods work to reduce the size of ComponentViews in memory.

Fixes #14172 and #14174.

Microsoft Reviewers: Open in CodeFlow

@acoates-ms acoates-ms requested review from a team as code owners December 9, 2024 18:36
Comment thread vnext/Microsoft.ReactNative/Fabric/ComponentView.cpp
@microsoft-github-policy-service microsoft-github-policy-service Bot added Area: Component Views New Architecture Broad category for issues that apply to the RN "new" architecture of Turbo Modules + Fabric labels Dec 9, 2024
@acoates-ms acoates-ms merged commit 91ad48e into microsoft:main Dec 9, 2024
@acoates-ms acoates-ms deleted the visualtoaddto branch December 9, 2024 21:45
acoates-ms added a commit to acoates-ms/react-native-windows that referenced this pull request Dec 9, 2024
…Into (microsoft#14182)

* Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto

* Change files

* format

* fix
acoates-ms added a commit to acoates-ms/react-native-windows that referenced this pull request Dec 9, 2024
…Into (microsoft#14182)

* Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto

* Change files

* format

* fix
acoates-ms added a commit to acoates-ms/react-native-windows that referenced this pull request Dec 9, 2024
…Into (microsoft#14182)

* Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto

* Change files

* format

* fix
acoates-ms added a commit that referenced this pull request Dec 10, 2024
…hildrenInto (#14187)

* Add linter exception to generated headers for Office builds (#14157)

* Add linter exception to generated headers for Office builds

* Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto (#14182)

* Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto

* Change files

* format

* fix

* fix

---------

Co-authored-by: Sarang Gokhale <sagokhal@microsoft.com>
acoates-ms added a commit that referenced this pull request Dec 10, 2024
…hildrenInto (#14188)

* Add linter exception to generated headers for Office builds (#14157)

* Add linter exception to generated headers for Office builds

* Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto (#14182)

* Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto

* Change files

* format

* fix

* change files

* fix

---------

Co-authored-by: Sarang Gokhale <sagokhal@microsoft.com>
acoates-ms added a commit that referenced this pull request Dec 10, 2024
…hildrenInto (#14186)

* Add linter exception to generated headers for Office builds (#14157)

* Add linter exception to generated headers for Office builds

* Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto (#14182)

* Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto

* Change files

* format

* fix

* fix

---------

Co-authored-by: Sarang Gokhale <sagokhal@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Component Views New Architecture Broad category for issues that apply to the RN "new" architecture of Turbo Modules + Fabric

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

Custom components should be able to override VisualToMountChildrenInto

2 participants